Skip to content

Conversation

ralphbean
Copy link

For https://issues.redhat.com/browse/KONFLUX-6210, clair needs access to a name and cpe label that it can use to look up the image in VEX statements.

See also release-engineering/rhtap-ec-policy#149

Signed-off-by: Ralph Bean [email protected]
Assisted-by: Gemini

…vability-console-plugin-pf4-zstream

For https://issues.redhat.com/browse/KONFLUX-6210, clair needs access to a name and cpe label that it can use to look up the image in VEX statements.

See also release-engineering/rhtap-ec-policy#149

Signed-off-by: Ralph Bean <[email protected]>
Assisted-by: Gemini
Copy link

openshift-ci bot commented Sep 17, 2025

Hi @ralphbean. Thanks for your PR.

I'm waiting for a netobserv member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@jotak
Copy link
Member

jotak commented Sep 18, 2025

Hi @ralphbean , thanks for your PR. I've added a commit to pass it as an arg, which is more consistent with our workflow.
Also, I assume we'll need to cherry-pick this change in our other release branches

Copy link

codecov bot commented Sep 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.81%. Comparing base (db928d3) to head (b28948e).
⚠️ Report is 1 commits behind head on release-1.9-pf4.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-1.9-pf4    #1026   +/-   ##
================================================
  Coverage            56.81%   56.81%           
================================================
  Files                  196      196           
  Lines                10204    10204           
  Branches              1186     1186           
================================================
  Hits                  5797     5797           
  Misses                4043     4043           
  Partials               364      364           
Flag Coverage Δ
uitests 59.80% <ø> (ø)
unittests 50.45% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jotak
Copy link
Member

jotak commented Sep 18, 2025

/lgtm
/approve

Copy link

openshift-ci bot commented Sep 18, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jotak

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jotak jotak added the ok-to-test To set manually when a PR is safe to test. Triggers image build on PR. label Sep 18, 2025
Copy link

New image:
quay.io/netobserv/network-observability-console-plugin:9b2f0c2

It will expire after two weeks.

To deploy this build, run from the operator repo, assuming the operator is running:

USER=netobserv VERSION=9b2f0c2 make set-plugin-image

@jotak
Copy link
Member

jotak commented Sep 18, 2025

/ok-to-test

@openshift-merge-bot openshift-merge-bot bot merged commit 4b02f8b into netobserv:release-1.9-pf4 Sep 18, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved lgtm ok-to-test To set manually when a PR is safe to test. Triggers image build on PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants